Skip to content

fix darwin rebuild - #22

Merged
phonkd merged 1 commit into
mainfrom
push-rkrsyzyzknwt
Jul 9, 2026
Merged

fix darwin rebuild#22
phonkd merged 1 commit into
mainfrom
push-rkrsyzyzknwt

Conversation

@phonkd

@phonkd phonkd commented Jul 9, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings July 9, 2026 16:45
@phonkd
phonkd merged commit 2fcbc9d into main Jul 9, 2026
2 checks passed
@phonkd
phonkd deleted the push-rkrsyzyzknwt branch July 9, 2026 16:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to fix macOS (nix-darwin) rebuilds by adjusting Home Manager configuration to avoid a failing assertion, and updates flake inputs to newer revisions.

Changes:

  • Disable programs.fzf.enableNushellIntegration in modules/shell.nix to avoid the Home Manager assertion that requires a newer fzf than what the pinned nixpkgs provides.
  • Adjust yt-dlp packaging in the desktop Home Manager modules (remove the custom override and add plain yt-dlp).
  • Update flake.lock inputs (notably home-manager, nix-darwin, and others) and simplify some input indirections.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
modules/shell.nix Disables fzf Nushell integration to avoid HM’s fzf version assertion during darwin rebuilds.
modules/desktop.nix Removes yt-dlp override usage and adds yt-dlp plainly to the nixos-specific home packages list.
flake.lock Updates pinned flake inputs and cleans up/simplifies some input references.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants